-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Task/WG-10: Add projects and user queries. #137
Conversation
…zmapper into task/WG-10-rtk-query
@@ -10,7 +10,9 @@ | |||
"test": "jest", | |||
"lint": "npm run lint:js", | |||
"lint:js": "eslint . --ext .js,.jsx,.ts,.tsx", | |||
"lint:fix": "eslint . --ext .js,.jsx,.ts,.tsx --fix" | |||
"lint:fix": "eslint . --ext .js,.jsx,.ts,.tsx --fix", | |||
"prettier:check": "prettier --single-quote --check src", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this need to be added to CI?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
adding a ticket: https://tacc-main.atlassian.net/browse/WG-197
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
@@ -10,7 +10,9 @@ | |||
"test": "jest", | |||
"lint": "npm run lint:js", | |||
"lint:js": "eslint . --ext .js,.jsx,.ts,.tsx", | |||
"lint:fix": "eslint . --ext .js,.jsx,.ts,.tsx --fix" | |||
"lint:fix": "eslint . --ext .js,.jsx,.ts,.tsx --fix", | |||
"prettier:check": "prettier --single-quote --check src", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
adding a ticket: https://tacc-main.atlassian.net/browse/WG-197
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Overview:
Adds basic project and user queries for react.
PR Status:
Related Jira tickets:
Summary of Changes:
Added basic query to userinfo and projects to production geoapi.
Also added slice for projects and modified
authSlice
to include user information.Added prettier in npm script.
Testing Steps:
UI Photos:
Notes:
I've tried setting locally with the jwt.. For some reason I get a CORS error for endpoints that should have that covered.